Regenerate overview catalogs after app 468 moved to src/00/00 - #696
Merged
Conversation
Commit 7a028fe moved z2ui5_cl_demo_app_468 from src/01/01 to src/00/00 but did not regenerate the two get_catalog( ) tables, so the generate_overview_apps check failed on standard. Per AGENTS.md section 4 the catalogs are a generated mirror of the folder tree, so 'npm run launchpad' moves the tile from demo_app_g00 (group 'Basic I') to sample_app_g01 (group 'extended'), matching the app's new subpackage CTEXT. abaplint: 0 issues, 743 files analyzed.
oblomov-dev
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes the failing
generate_overview_appscheck onstandard.Commit 7a028fe ("reassign package") moved
z2ui5_cl_demo_app_468fromsrc/01/01tosrc/00/00, but did not regenerate the twoget_catalog( )tables. Per AGENTS.md §4 the catalogs are a generated mirror of the folder
tree, and the workflow enforces that by running
npm run launchpadandfailing on any diff — so
standardhas been red since that commit.This PR contains only the generator output. The package reassignment itself
already happened in 7a028fe and is not changed here.
Changes
Produced by
npm run launchpad, not hand-edited:src/01/z2ui5_cl_demo_app_g00.clas.abap— tile removed (was groupBasic I)src/00/z2ui5_cl_sample_app_g01.clas.abap— tile added (groupextended)The new group matches the CTEXT of the app's new subpackage
src/00/00.Verification
npm run launchpad— now diff-freenpx abaplint— 0 issues, 743 files analyzednpm run check:agents— tree matches src/ (21 subpackages)Note
Not changed here, but worth a look: AGENTS.md §2 rule 6 maps
"Experimental / work-in-progress" to
00/07rather than00/00. The otherEXPERIMENTALapps (319, 313, 314) already sit in00/00, so the placement isconsistent with what's there — flagging it rather than overriding it.